Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2.4.2リリース準備 #1872

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

v2.4.2リリースのため

カテゴリ

  • リリース準備作業

PR の背景

v2.4.2リリースの準備のために変更の取り込みを行います。

PR のメリット

PR のデメリット (トレードオフとかあれば)

仕様・動作説明

v2.4.2ブランチ(内容はv2.4.1)に最新のmasterの変更をマージします。

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

参考資料

berryzplus and others added 30 commits October 12, 2021 00:20
…ring_version_info

バージョン情報をリファクタリングする
…-rev-list_to_count_commits

累積コミット数の取得にgit rev-listを使う
…wsx_h

Windows 10 SDKのメッセージクラッカーを使えるようにする
CEditWndのインスタンスにはグローバル変数のようにアクセスできる。このため、コンストラクタでCEditWndのインスタンスを要求してクラス内外でのアクセスに流用する構造は無駄と考えられる。
置換パターン
\b((m_)?pc?)?(Edit)?View(Dst)?->m_pcEditWnd->
\b((m_)?pc?)?(Edit)?Doc(Ref)?->m_pcEditWnd->
CEditWndのインスタンスはクラスのstatic変数に保持されるので、グローバル変数に持つと二重保持になり無駄です。
…ence_of_CEditWnd

編集ウインドウへの参照をリファクタリングする
変数宣言のconstを削除し、c style-castしなくて良いようにする
…mapview

ミニマップの実装クラスを分割する
…BlkRedawCount をインクリメントする処理を追加

キャレット移動の CMoveCaretOpe 追加では再描画は必要無いはず
GermanAizek and others added 13 commits July 18, 2022 17:59
Shiftを押さなかった場合にスクロールしなくなってしまっていたのを修正。
Code modernize C++17, refactor and minor optimize
CMakeLists.txtのコメントタイポ修正
マルチユーザー設定ファイル(sakura.exe.ini)のテストが失敗したときに、テストのために作成したexe.iniが削除されず、他ケースが失敗していたのを修正
マルチユーザー設定ファイルのUserRootFolderに
2:ドキュメントフォルダ、3:デスクトップを指定した場合、
OneDriveのフォルダが取得されてしまう不具合を修正する。

この不具合はOneDriveをセットアップしないと再現しないのでCI/CDではチェックできない。
…t_for_exeini

マルチユーザー設定ファイル(sakura.exe.ini)のテストケース修正
…ini_settings

マルチユーザー設定ファイルの不具合修正
@berryzplus berryzplus mentioned this pull request Nov 30, 2022
9 tasks
@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

マージミスがあったようなので対応できるまでdraftにします。

@berryzplus berryzplus marked this pull request as draft December 1, 2022 01:07
@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

@berryzplus berryzplus marked this pull request as ready for review December 1, 2022 02:42
@beru beru added IMPORTANT 早急に解消すべきもの Release Release作業チケット【ChangeLog除外】 labels Dec 2, 2022
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

x64版のインストーラーをダウンロードしてインストール後に起動して簡単な操作を行いました。

あまり動作確認をしっかりしていませんがリリースして問題無いと思います。
もし問題が見つかったら後で v2.4.2.1 をリリースしてカバーすれば良いかなと…。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージして #1870 を進めます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IMPORTANT 早急に解消すべきもの Release Release作業チケット【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.